Skip to content

800: Upgrade to java 25, upgrade vulnerable dependencies#801

Merged
Bert-R merged 1 commit into
obsidiandynamics:masterfrom
ckorakidis:feature/upgrade-to-java-25
Dec 15, 2025
Merged

800: Upgrade to java 25, upgrade vulnerable dependencies#801
Bert-R merged 1 commit into
obsidiandynamics:masterfrom
ckorakidis:feature/upgrade-to-java-25

Conversation

@ckorakidis
Copy link
Copy Markdown
Contributor

  • Upgrade to Java 25
  • Upgrade vulnerable dependencies
  • Upgrade images
  • Fix error page parsing

Copy link
Copy Markdown
Collaborator

@Bert-R Bert-R left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ckorakidis Thanks!
Looks good, except for one small thing.

Comment thread src/main/docker/Dockerfile Outdated
@@ -1,4 +1,4 @@
FROM eclipse-temurin:17.0.17_10-jdk
FROM eclipse-temurin:25
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Set a specific version, so we know what version we run:

Suggested change
FROM eclipse-temurin:25
FROM eclipse-temurin:25.0.1_8-jdk

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure @Bert-R, done

@Bert-R
Copy link
Copy Markdown
Collaborator

Bert-R commented Dec 13, 2025

@davideicardi Can you review this one too? Are you OK upgrading to Java 25, the latest LTS release?

@ckorakidis ckorakidis force-pushed the feature/upgrade-to-java-25 branch from 0049d40 to 21fb887 Compare December 13, 2025 22:19
Copy link
Copy Markdown
Collaborator

@davideicardi davideicardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok for me!

@Bert-R

Copy link
Copy Markdown
Collaborator

@Bert-R Bert-R left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ckorakidis Thanks for the PR and for solving the other issues alongside the Java upgrade!

@Bert-R Bert-R merged commit 43cfea3 into obsidiandynamics:master Dec 15, 2025
1 check passed
@davideicardi
Copy link
Copy Markdown
Collaborator

@Bert-R I think to release a new major release, 5.0. What do you think?

@Bert-R
Copy link
Copy Markdown
Collaborator

Bert-R commented Dec 18, 2025

Not yet. @ckorakidis reported a major issue: #803

My first thought was to adopt Spring Boot 4 and OpenAPI starter 3 first, but that is not trivial. Kafdrop uses Undertow and that's not supported in Spring Boot anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants